feat: #79 하루 마감 JSON API (daily_plan_finalize) 연결 - #80

138개 테스트 전부 통과 확인했고(python manage.py test planner), recovery_group_id가 maintain_volume/core_focus 양쪽에 동일하게 부여되는 것도 generate_recovery_options() 코드로 직접 확인했습니다 — or 체이닝 방식이 정확합니다.

다만 _get_today_daily_plan()의 조회 순서에 실질적인 문제가 있어 보입니다.

python
def _get_today_daily_plan(user):
exam_period = (
ExamPeriod.objects.filter(user=user, status=ExamPeriodStatus.ACTIVE)
.order_by('-created_at').first()
)
if exam_period is None:
return None
return DailyPlan.objects.filter(
exam_period=exam_period, date=timezone.localdate()
).first()

시험기간을 먼저 "가장 최근 생성된 ACTIVE 것 하나"로 고정한 뒤, 그 안에서만 오늘 날짜 계획을 찾습니다.

프로젝트 전체에 ExamPeriod를 COMPLETED로 전환하는 코드가 없어서(grep -rn "status = ExamPeriodStatus.COMPLETED" 결과 없음), 사용자가 시험기간을 여러 개 만들면 전부 계속 ACTIVE로 남습니다. 이 상태에서 더 오래된 시험기간 쪽에 오늘 날짜 계획이 있어도, 최근 시험기간 안에 오늘 계획이 없으면 바로 None을 반환해 404가 뜹니다. 실제로는 마감할 계획이 분명히 있는데도요.

대시보드 쪽에서 비슷한 패턴을 이미 한 번 지적드린 적 있는데(#61), 그때는 "화면에 뭘 보여줄지"의 문제였다면 이번엔 마감이라는 되돌릴 수 없는 액션 자체를 시작하지 못하게 막는 문제라 더 실질적입니다.

제안: 시험기간을 먼저 고르지 말고 오늘 날짜인 DailyPlan을 기준으로 바로 조회하면 이 문제 자체가 없어질 것 같습니다.

python
def _get_today_daily_plan(user):
return DailyPlan.objects.filter(
exam_period__user=user,
exam_period__status=ExamPeriodStatus.ACTIVE,
date=timezone.localdate(),
).order_by('-exam_period__created_at').first()

관련해서 아래 테스트도 추가되면 좋겠습니다.

사용자가 ACTIVE 시험기간을 2개 이상 갖고 있고, 더 오래된 시험기간에만 오늘 날짜 DailyPlan이 있는 경우에도 정상적으로 마감되는지

이 부분 반영되면 나머지는 로직(트랜잭션, recovery_group_id, 에러 응답 매핑)까지 다 확인했고 문제없어 보입니다.

참고로 별개 사항인데, 직전에 리뷰했던 PR #78(progress_record)은 isSuccess 구조를 폐기하고 flat 구조로 가기로 했는데 이 PR은 isSuccess로 감싸는 구조를 씁니다. 두 PR을 같이 정리하실 때 API 응답 구조도 하나로 통일하는 걸 검토해주시면 좋겠습니다.

리뷰 감사합니다. 요청해주신 사항 반영했습니다.

• 최신 dev 병합 후 planner/views.py, planner/tests.py 충돌 해결
• #78의 progress_record 구현 및 테스트 유지
• ACTIVE 시험기간을 먼저 하나 선택하지 않고, 로그인 사용자의 ACTIVE 시험기간 중 오늘 날짜 DailyPlan을 직접 조회하도록 수정
• 최근 ACTIVE 시험기간에는 오늘 계획이 없고, 이전 ACTIVE 시험기간에만 오늘 계획이 있는 경우의 테스트 추가
• progress_record와 동일하게 하루 마감 API 응답을 flat JSON 구조로 통일

검증 결과:

python manage.py test planner.tests.ProgressRecordViewTests -v 2
# 12개 통과

python manage.py test planner.tests.DailyPlanFinalizeViewTests -v 2
# 11개 통과

python manage.py test planner
# 151개 통과

python manage.py check
# 이상 없음

python manage.py makemigrations --check
# No changes detected

재확인 부탁드립니다.